Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

15.0 mig delivery_ups_oca #991

Open
wants to merge 38 commits into
base: 15.0
Choose a base branch
from
Open

15.0 mig delivery_ups_oca #991

wants to merge 38 commits into from

Conversation

jans23
Copy link

@jans23 jans23 commented Feb 20, 2025

Successor of #986.

hbrunn and others added 30 commits February 20, 2025 15:13
@hbrunn @jans23
[ADD] delivery_carrier_label_ups
2053659
@StefanRijnhart @jans23
[RFR] Extract Test classes from Case classes for easier overriding
ea65ce0
@astirpe @jans23
[FIX] access to ir.config_parameter
3d27729
@victoralmau @jans23
[IMP] delivery_carrier_label_ups: black, isort, prettier
6015385
@victoralmau @jans23
[MIG] delivery_ups_oca: Migration to 13.0 + Refactor
21a8055
@oca-travis @jans23
[UPD] Update delivery_ups_oca.pot
a87e352
@OCA-git-bot @jans23
[UPD] README.rst
b80e84e
@victoralmau @jans23
[FIX] delivery_ups_oca: Prevent error when convert price in different…
d7b53a5 
… currency.
@OCA-git-bot @jans23
delivery_ups_oca 13.0.1.0.1
a65b7cf
@victoralmau @jans23
[FIX] delivery_ups_oca: Define the total weight correctly in an order.
9b89708
@OCA-git-bot @jans23
delivery_ups_oca 13.0.1.0.2
20c86b3
@victoralmau @jans23
[REF] delivery_ups_oca: Code refactor.
513395b
@oca-travis @jans23
[UPD] Update delivery_ups_oca.pot
c222ccc
@OCA-git-bot @jans23
delivery_ups_oca 13.0.1.0.3
51e43c0
@victoralmau @jans23
[FIX] delivery_ups_oca: Pevent errors in tests.
b03b116
@OCA-git-bot @jans23
delivery_ups_oca 13.0.1.0.4
1c76bc0
@victoralmau @jans23
[FIX] delivery_ups_oca: Show the errors to avoid the error of access …
6bfa2dd 
…to a non-existing key in the code.

TT37248
@OCA-git-bot @jans23
delivery_ups_oca 13.0.1.0.5
49d914e
@victoralmau @jans23
[FIX] delivery_ups_oca: Create label from create_shipping response to…
2c0dec2 
… avoid error.

TT37248
@victoralmau @jans23
[FIX] delivery_ups_oca: Set the correct URLs.
5982220
@victoralmau @jans23
[FIX] delivery_ups_oca: Apply domain to ups_default_packaging_id fiel…
9fa0865 
…d to show only UPS records.

TT37248
@victoralmau @jans23
[FIX] delivery_ups_oca: Set the correct weight if we use packages.
a907535 
TT37180
@victoralmau @jans23
[FIX] delivery_ups_oca: Fix packages when package_ids is not set.
01db2ba 
TT37248
@victoralmau @jans23
[FIX] delivery_ups_oca: Set correct ShipFrom (warehouse or company ad…
71fd0e3 
…dress).
@victoralmau @jans23
[IMP] delivery_ups_oca: Improve tests to allow inheritance.
9b65002 
TT37248
@jans23
[UPD] Update delivery_ups_oca.pot
0254c8a
@OCA-git-bot @jans23
delivery_ups_oca 13.0.2.0.0
ac38ae4
@victoralmau @jans23
[FIX] delivery_ups_oca: Fix cancel_shipment method
4da7fc9 
TT40776
@OCA-git-bot @jans23
delivery_ups_oca 13.0.2.0.1
d06be7d
@OCA-git-bot @jans23
[UPD] README.rst
7ae8cbc
@jans23 jans23 mentioned this pull request Feb 20, 2025
Closed
@jans23 jans23 force-pushed the 15.0-mig-delivery_ups_oca branch 2 times, most recently from 464e6db to d2d58bc Compare February 20, 2025 21:26
@rousseldenis
Copy link
Contributor

/ocabot migration delivery_ups_oca

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Feb 21, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 21, 2025
Open
32 tasks
@jans23 jans23 force-pushed the 15.0-mig-delivery_ups_oca branch 4 times, most recently from 75a1875 to 3296c5d Compare February 24, 2025 14:55
Tisho99
Tisho99 reviewed Feb 26, 2025
View reviewed changes
@jans23 jans23 force-pushed the 15.0-mig-delivery_ups_oca branch from 0886233 to 9ec5923 Compare February 28, 2025 20:28
@jans23 jans23 force-pushed the 15.0-mig-delivery_ups_oca branch from 9ec5923 to 180ac8d Compare March 1, 2025 19:05
Tisho99
Tisho99 approved these changes Mar 5, 2025
View reviewed changes
Copy link
Contributor

@Tisho99 Tisho99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Code review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tisho99 Tisho99 Tisho99 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.